Skip to content

fix: replace pushd, popd with cd #61

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 22, 2025
Merged

Conversation

CodeChenL
Copy link
Contributor

Improve script compatibility

@CodeChenL CodeChenL requested a review from RadxaYuntian as a code owner April 28, 2025 13:24
@CodeChenL CodeChenL requested a review from RadxaYuntian May 8, 2025 03:50
Copy link
Contributor

@RadxaYuntian RadxaYuntian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

squash

RadxaYuntian
RadxaYuntian previously approved these changes May 8, 2025
@RadxaYuntian RadxaYuntian enabled auto-merge May 8, 2025 06:03
auto-merge was automatically disabled May 9, 2025 08:21

Head branch was pushed to by a user without write access

@CodeChenL
Copy link
Contributor Author

吔?我还以为合并了,又提了几个commit准备提pr的

@RadxaYuntian
Copy link
Contributor

所以要一个pr对应一个分支啊
你先把这里的修复了

@RadxaYuntian
Copy link
Contributor

你要测试这些单独开个分支测试,现在这个PR无法继续推进

@CodeChenL
Copy link
Contributor Author

CodeChenL commented May 19, 2025

这边等#64 合并再整理触发CI

@RadxaYuntian RadxaYuntian enabled auto-merge May 22, 2025 09:33
@RadxaYuntian RadxaYuntian added this pull request to the merge queue May 22, 2025
Merged via the queue into RadxaOS-SDK:main with commit 931d6d8 May 22, 2025
47 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants